- User Since
- Feb 6 2020, 4:22 AM (189 w, 2 d)
Mar 24 2022
Added a check that an instruction is inlined before to try to find inlined CUs.
Mar 22 2022
Reduce code duplication.
Mar 21 2022
Aug 26 2021
Aug 17 2021
Feb 25 2020
I'm closing this review as current patch is not fixing the issue properly.
Feb 18 2020
When reading the following definition of method 'MaybeAlign Value::getPointerAlignment(const DataLayout &DL) const' most users assume that Value won't be modified by this method. If we don't guarantee it then it would be nice to remove const-qualifier and change name on something like 'accountPointerAlignment'. As renaming is less possible then a comment might be present that this method may change an IR.
Feb 17 2020
Feb 6 2020
We faced some side effect of this patch. During processing of getelementptr instruction it may create new user of this instruction. However this user is not actually used by generated code.
As example, during processing of the following instruction
%3 = load %struct_t*, %struct_t** getelementptr inbounds ([8 x %struct_t*], [8 x %struct_t*]* @globvar, i64 0, i64 0), align 8, !tbaa !16
the following constant expresion is created