[PATCH] D21110: [CFLAA] Add yet another StratifiedAttr

George Burgess IV via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 16:46:23 PDT 2016


george.burgess.iv added a comment.

> I (mistakenly) thought AttrGlobal/Arg was mainly motivated by the loads from globals/args, not the values. Now that you've explained your thoughts and I have a better understanding, I think your plan is definitely superior :)


Ehh, it was initially motivated by the values, but turned into a very convenient (albeit unclear) way to handle the loads, as well. Looking back, I could've done a much better job of splitting it all out properly, rather than going with the easiest solution. :P


http://reviews.llvm.org/D21110





More information about the llvm-commits mailing list