efriedma added a comment.

We usually prefer to generate error messages for incorrect parameters to 
builtins in SemaChecking.cpp.

I don't think there's really an infinite loop here.  The parameter to 
__builtin_frame_address is an unsigned integer; values greater than 0x7FFFFFFU 
aren't special.  The reason it takes a very long time is that we try to unroll 
the implied loop; generating an object file with four billion load instructions 
takes essentially forever.  We could impose a smaller limit, but it would be 
sort of arbitrary.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66839/new/

https://reviews.llvm.org/D66839



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to