[llvm-commits] [llvm] r49243 - /llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
dalej at apple.com
Mon Apr 7 09:52:16 PDT 2008
On Apr 7, 2008, at 9:49 AM, Evan Cheng wrote:
> Ok, can you add a "set root to first pending load and flush" function
> when you have a chance?
No, as I said, I do not believe adding a new function is a good idea.
If you want to change something here the right thing is to rename
getRoot and getControlRoot to something like flushXXXandAdvanceRoot.
> On Apr 5, 2008, at 10:15 AM, Dale Johannesen wrote:
>> On Apr 5, 2008, at 1:57 AM, Evan Cheng wrote:
>>> Is this fixing a regression from Dan's patch? Can you separate out
>>> flushing functionality from getroot? It seems a bit strange to use
>>> getRoot for its side effects.
>> Well, things named getXXX should not have side effects, IMO. If I
>> were going
>> to work on naming I would probably rename getRoot and getControlRoot
>> to something
>> more descriptive rather than add a new function. But in general I
>> prefer minimal
>> changes to more invasive ones. (I wasn't familiar with this code
>> Glad this fixed some Ada regressions, not too surprising.
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
More information about the llvm-commits