fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!

> The __builtin_align_{up,down} code generation could also make use of this 
> IIRC. I think the reason I didn't do this initially was concerns about 
> ptrmask not being optimized as well.

Yeah, I think it would probably be good to use the intrinsic and just address 
the remaining relevant gaps, if there are any.


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

https://reviews.llvm.org/D142934

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D142934: clang: Us... Florian Hahn via Phabricator via cfe-commits

Reply via email to